Skip to content

Makecheck debuginfo arm linux androideabi #10261

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

yichoi
Copy link
Contributor

@yichoi yichoi commented Nov 4, 2013

In order to pass make check on android target

credit to @ksh8281

@ksh8281
Copy link
Contributor

ksh8281 commented Nov 4, 2013

in order to debuginfo test of make check(arm-linux-androideabi)

@@ -219,6 +219,19 @@ pub fn mode_str(mode: mode) -> ~str {
}

pub fn run_tests(config: &config) {
if config.target.equals(&~"arm-linux-androideabi"){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there some problem with config.target == ~"arm-linux-androideabi", or (preferably) "arm-linux-androideabi" == config.target?

@huonw
Copy link
Member

huonw commented Nov 4, 2013

There appears to be an empty file added: src/test/run-fail/rt-set-exit-status-fail.

Also, the rust convention is for spacing like loop {, do task::try { and if condition { etc (rather than loop{, do task::try{ and if condition{).

}

if ProcRes.status != 0 {
fatal(~"gdb failed to execute");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation

@brson
Copy link
Contributor

brson commented Nov 5, 2013

Nice work @ksh8281! I love improvements to the test suite.

@huonw
Copy link
Member

huonw commented Nov 5, 2013

This needs to be rebased on top of master again.

bors added a commit that referenced this pull request Nov 6, 2013
…roideabi, r=brson

In order to pass make check on android target

credit to @ksh8281
@bors bors closed this Nov 6, 2013
@bors bors merged commit 4e54828 into rust-lang:master Nov 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants